Skip to content

Bug: Inconsistency definition of the correlation_id #480

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
albert-cg opened this issue Apr 14, 2025 · 1 comment
Open

Bug: Inconsistency definition of the correlation_id #480

albert-cg opened this issue Apr 14, 2025 · 1 comment

Comments

@albert-cg
Copy link

Describe the bug
Inconsistency in the definition of the correlation_id attribute in two tables. Same attribute but different length.

  • postgres-persistence/src/main/resources/db/migration_postgres/V8__indexing.sql

CREATE TABLE workflow_index (
workflow_id VARCHAR(255) NOT NULL,
correlation_id VARCHAR(128) NULL,
workflow_type VARCHAR(128) NOT NULL,
start_time TIMESTAMP WITH TIME ZONE NOT NULL,
status VARCHAR(32) NOT NULL,
json_data JSONB NOT NULL,
PRIMARY KEY (workflow_id)
);

  • postgres-persistence/src/main/resources/db/migration_postgres/V1__initial_schema.sql

CREATE TABLE workflow (
id SERIAL,
created_on TIMESTAMP DEFAULT CURRENT_TIMESTAMP,
modified_on TIMESTAMP DEFAULT CURRENT_TIMESTAMP,
workflow_id varchar(255) NOT NULL,
correlation_id varchar(255),
json_data TEXT NOT NULL,
PRIMARY KEY (id)
);

A clear and concise description of what the bug is.

Details
Conductor version: 3.21.12
Persistence implementation: Postgres
Queue implementation: Postgres
Lock: N/A
Workflow definition: N/A
Task definition: N/A
Event handler definition: N/A

Expected behavior
Same type of definition

@albert-cg albert-cg changed the title Postgres creation Bug: Inconsistency definition of the correlation_id Apr 14, 2025
@JCHacking
Copy link
Contributor

It would also be good to review the scripts of the rest of the persistence.
I think the best thing to do is to create a new migration with the highest value in order not to break anything.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants